Workflow [PR], commit [24d1ace]

@blau-ai

CI triage for #2129

Verdict: 8 failing checks → 1 PR-caused (a clang-tidy nit with a one-line fix), 7 pre-existing/infra. The only thing this PR actually broke is the arm_tidy lint. Everything else fails the same way on the base branch antalya-26.6 (run 30468399078 @ 4359a07, which is this PR's merge base) or is a runner/memory infra flake.

🔴 PR-caused (1) — needs a fix

Build (arm_tidy) — job log

src/Processors/Formats/Impl/Parquet/PrepareForWrite.cpp:504:20: error: variable 'converted_scale' is not initialized
  [cppcoreguidelines-init-variables,-warnings-as-errors]
  504 |             UInt32 converted_scale;

arm_tidy builds with WERROR=ON, so the clang-tidy cppcoreguidelines-init-variables diagnostic is fatal. In practice converted_scale is always assigned before use (the else throws), so this is a lint-only issue — it does not affect the functional binary — but it blocks the tidy gate.

Suggested fix (one line):

-            UInt32 converted_scale;
+            UInt32 converted_scale = 0;

Note: every non-tidy Build (...) job showed skipping (build-cache reuse), so arm_tidy was the only job that actually compiled the new code — this is the single blocker introduced by the diff.

🟡 Pre-existing on the base branch (5) — not caused by this PR

Confirmed by running the same suites on antalya-26.6 @ 4359a07 (the merge base), where they already fail:

🟠 Infra / flaky (2) — safe to re-run

• SQLLogic test (passed on base) — not a correctness regression. The server hit its global memory limit and every query cascaded:

  DB::Exception: (total) memory limit exceeded: would use 3.00 GiB ... current RSS: 25.96 GiB, maximum: 3 GiB
  

  RSS 26 GiB against a 3 GiB limit → the 190,317 "failed tests" are all this one memory-limit cascade, and "total tests < minimum" is the run aborting early. Environmental; re-run should recover.
• Stress test (arm_debug) — failure signature is Timeout exceeded while receiving data from server / Connection refused (localhost:9000) under load. Classic stress flakiness (base failed Stress test (arm_tsan) similarly); nothing in the diff touches server core. Re-run.

Bottom line

The Time64 / Iceberg-time change looks clean w.r.t. CI — no new test failures are attributable to it. Only the arm_tidy lint needs the one-line init above; the rest are pre-existing base-branch redness or infra flakes that a re-run (or a base-branch fix) will clear.

I have the one-line fix ready. How would you like it delivered — a direct commit to feature/antalya-26.6/pr-1761, or a separate PR from a blau/* branch? I won't push until you say which.

_{🤖 Generated with Claude Code}